-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated the environment docs with the database and extension #1188
Conversation
Signed-off-by: Harshad Reddy Nalla <[email protected]>
Ready for review again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! 💯 👏🏼
List of installed extensions | ||
Name | Version | Schema | Description | ||
--------------------+---------+------------+----------------------------------------------------------- | ||
amcheck | 1.0 | public | functions for verifying relation integrity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this somethign we have declared in some yaml? or can put to a file in a repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can set this from a component that talks to storage.
so we will have this in init-job, so some newly setup their environment can benefit from it.
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: harshad16 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…-station#1188) Signed-off-by: Harshad Reddy Nalla <[email protected]>
updated the environment docs with the database and extension
Signed-off-by: Harshad Reddy Nalla [email protected]
Related Issues and Dependencies
Related-to:
#726
#725
Does this require new deployment ?
Not needed
Description
document update with information on how to add extension to database.